-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(fe): create admin instructor management page #2341
Open
jwoojin9
wants to merge
10
commits into
main
Choose a base branch
from
t1215-add-instructor-management-sidebar
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jwoojin9
added
⛳️ team-frontend
💡 feature-admin
Features for admin & group manager
🌊 squad-__init__
labels
Jan 28, 2025
The preview deployment is ready. 🟢 Open Preview | Open Build Logs Last updated at: 2025-01-30 18:18:11 CET |
파일들이 워낙 많긴 한데 기존에 있던 admin 파일들에서 복붙한 게 아주아주 많습니다...! 특히 assignment 파일은 contest에서 이름만 바꾼 수준... 어쨌든 이렇게 말하면 좀 잘못된 것 같긴 한데 group 관련 아니면 좀 대충 보셔도 될 것 같아요!! |
jwoojin9
requested review from
Kohminchae,
juhyeong0505,
eunnbi,
B0XERCAT and
mingyu27
January 30, 2025 18:31
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
초안입니다! role을 확인해서 group sidebar를 결정하는 로직이 필요할 것 같고, notice, exam 등 기타 세부 페이지는 백엔드 작업 완료되면 착수할 예정입니다. Group 사이드바를 2개로 한 것은 그냥 기존 Contest overall에서 사용하던 tab 방식보다 편할 것 같았기 때문입니다. 사용할 수 있는 공간이 가로로는 줄지만 세로로는 늘어나겠죠? 물론 나중에 디자인팀이 시키는 대로 바꾸겠지만 지금 더 바쁜 일이 많다보니 제 임의로 만들어 봤습니다. (+접고 펼치는 버튼을 추가했어요!)
근데 이렇게 놓고 보니 Group 밑에 여러 개가 나오는 게 좀 조잡하긴 하네요... 그래도 제 생각엔 Codedang을 사용하는 교수님 한 명의 Course가 딱히 많지도 않을 테도 Group 누르고 테이블에서 한 번 더 누를 필요 없이 한 번에 가고 싶은 Course로 이동하는 게 교수/조교님 입장에서 더 편할 것 같긴 합니다..!
closes TAS-1215
P.S. 은비 누나, datatable 리팩토링 진짜 미친 것 같애요!! 이렇게 편하게 갖다 쓸 수가 ㄷㄷ...
에러 해결해주신 민채 누나도 너무 감사합니다...🙏🙏
Additional context
백엔드 진행상황이 어느 정도인진 모르겠지만 일단 가장 필요하다고 판단되는 요청사항들입니다
groupAdmin role을 가진 샘플 계정 생성(groupAdmin인 경우 sidebar를 다르게 표시하기 위함)
그 groupAdmin이 어떤 group들의 admin인지 groupId를 보관하는 리스트
일반 학생들이랑 조교들이 정확히 어떻게 group에 가입하는지 정책 이해를 못했는데 어쨌든 관련 API
Problem에 있는 Get Contest Problems, Update ContestProblems Order/Scores 같은 것도 Assignment 버전으로 추가
submission도 마찬가지
Group에 Professor, Semester, Contact 등 세부 정보 추가(근데 이러면 group이랑 course랑 분리해야 되는 거 아니에요?)
userId와 groupId로 해당 유저의 assignment 점수들 가져오는 API
특정 그룹에 대한 notices 가져오기(생성은 있어요), 댓글 기능 추가
Q&A
notice 댓글이나 Q&A 같은 경우에는 읽었는지 여부를 기록해서 새로운 알림을 빨간 점과 개수 등으로 표시해주는 기능도 좋을 것 같아요!
이거 말고도 좀 더 있었던 것 같은데 까먹었어요...ㅎㅎ
Before submitting the PR, please make sure you do the following
fixes #123
).